Split original ASN.1 code from asn.c into asn_orig.c#9920
Split original ASN.1 code from asn.c into asn_orig.c#9920dgarske wants to merge 2 commits intowolfSSL:masterfrom
Conversation
There was a problem hiding this comment.
Pull request overview
Moves the non-template ASN.1 implementation into a separate asn_orig.c file and ensures it’s included in distributed source archives.
Changes:
- Adds the new
wolfcrypt/src/asn_orig.cfile to Automake’sEXTRA_DISTso it’s shipped in release tarballs.
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
You can also share your feedback on Copilot code review. Take the survey.
|
Jenkins retest this please |
|
Jenkins retest this please |
2 similar comments
|
Jenkins retest this please |
|
Jenkins retest this please |
Move non-template (WOLFSSL_ASN_ORIGINAL) code into asn_orig.c and include from asn.c.
Add settings.h include before the compile guard in asn_orig.c so WOLFSSL_IGNORE_FILE_WARN is available when compiled separately. Add -Wno-pedantic for asn_orig.c in async examples Makefile to suppress empty translation unit warning.
There was a problem hiding this comment.
Pull request overview
Copilot reviewed 2 out of 4 changed files in this pull request and generated no new comments.
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
You can also share your feedback on Copilot code review. Take the survey.
|
Jenkins retest this please |
Move non-template (
WOLFSSL_ASN_ORIGINAL) code intoasn_orig.cand include fromasn.c.